This makes the showAndLogExceptionWithTelemetry function callable without depending on the vscode module. This makes it easier to test and mock.

Please review this PR commit-by-commit.

Unfortunately, most of the call sites are still using telemetryListener as argument rather than app.telemetry. I'm hoping that this will be fixed in the future as we move more to using the App.

Checklist

• CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
• Issues have been created for any UI or other user-facing changes made by this pull request.
• [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.